Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Switch to uv and ruff #86

Merged
merged 5 commits into from
Nov 29, 2024

Conversation

phoenixpereira
Copy link
Member

@phoenixpereira phoenixpereira commented Nov 28, 2024

Description

Use uv as a package and project manager and Ruff as a linter and code formatter.

Changes Made

  • Replace Poetry with uv as the package and project manager
  • Replace Black with Ruff as a linter and code formatter
  • Update Dockerfile and workflows to use uv and Ruff
  • Update README
  • Add Git Hook for Ruff

Related Issues

Fixes #80. Replaces the need for the following, so fixes #27 and fixes #20. Also closes #38 since that is no longer needed.

Additional Notes

N/A

@phoenixpereira phoenixpereira self-assigned this Nov 28, 2024
@phoenixpereira phoenixpereira added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels Nov 28, 2024
@phoenixpereira phoenixpereira marked this pull request as ready for review November 29, 2024 07:11
@phoenixpereira phoenixpereira merged commit 0b3f3ef into compsci-adl:main Nov 29, 2024
2 checks passed
@phoenixpereira phoenixpereira deleted the chore/uv-ruff branch November 29, 2024 07:50
@tinnamchoi
Copy link
Contributor

I can't believe you didn't wait for me to finish my PR this is so sad :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation
Projects
Status: Done
2 participants